Skip to content

GH-50744: [R] Add read_ipc_file and write_ipc_file to _pkgdown.yml reference index - #50745

Merged
raulcd merged 1 commit into
apache:mainfrom
thisisnic:GH-50744-fixdocs
Jul 31, 2026
Merged

GH-50744: [R] Add read_ipc_file and write_ipc_file to _pkgdown.yml reference index#50745
raulcd merged 1 commit into
apache:mainfrom
thisisnic:GH-50744-fixdocs

Conversation

@thisisnic

@thisisnic thisisnic commented Jul 30, 2026 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

Rationale for this change

Errors due to functions missing from pkgdown docs

What changes are included in this PR?

Adds them

Are these changes tested?

Will run CI

Are there any user-facing changes?

No

@thisisnic
thisisnic requested a review from jonkeane as a code owner July 30, 2026 16:06
Copilot AI review requested due to automatic review settings July 30, 2026 16:06
@thisisnic thisisnic changed the title GH-50744: [R] Add read_ipc_file and write_ipc_file to _pkgdown.yml reference index # GH-50744: [R] Add read_ipc_file and write_ipc_file to _pkgdown.yml reference index Jul 30, 2026
@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Jul 30, 2026
@thisisnic

thisisnic commented Jul 30, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit -g docs

Copilot AI reviewed Jul 30, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the R package’s pkgdown reference index so read_ipc_file() and write_ipc_file() appear in the generated documentation site, addressing missing-function doc errors.

Changes:

  • Added read_ipc_file to the “Read files” reference section.
  • Added write_ipc_file to the “Write files” reference section.

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown 
Invalid group(s) {'docs'}. Must be one of {'ruby', 'fuzz', 'python', 'vcpkg', 'homebrew', 'wheel', 'nightly-packaging', 'verify-rc-source', 'packaging', 'example', 'nightly', 'verify-rc-source-macos', 'cpp', 'verify-rc', 'example-python', 'integration', 'c-glib', 'nightly-tests', 'verify-rc-source-linux', 'r', 'test', 'nightly-release', 'example-cpp'}
The Archery job run can be found at: https://github.com/apache/arrow/actions/runs/30559987190

@raulcd

raulcd commented Jul 30, 2026

Copy link
Copy Markdown
Member

@github-actions crossbow submit preview-docs

@raulcd

raulcd commented Jul 30, 2026

Copy link
Copy Markdown
Member

@github-actions crossbow submit test-debian-13-docs test-r-devdocs

@raulcd

raulcd commented Jul 30, 2026

Copy link
Copy Markdown
Member

Probably those last two better than preview-docs :)

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

Revision: b1e64ce

Submitted crossbow builds: ursacomputing/crossbow @ actions-77b235368e

Task Status
test-debian-13-docs GitHub Actions
test-r-devdocs GitHub Actions

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown

Revision: b1e64ce

Submitted crossbow builds: ursacomputing/crossbow @ actions-8fff552a4b

Task Status
preview-docs GitHub Actions

raulcd
raulcd approved these changes Jul 31, 2026
View reviewed changes

@raulcd raulcd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @thisisnic

@raulcd
raulcd merged commit d735dc9 into apache:main Jul 31, 2026
35 of 36 checks passed
@raulcd raulcd removed the awaiting committer review Awaiting committer review label Jul 31, 2026
@raulcd raulcd mentioned this pull request Jul 31, 2026
Closed
@github-actions github-actions Bot added the awaiting merge Awaiting merge label Jul 31, 2026
@conbench-apache-arrow

conbench-apache-arrow Bot commented Aug 2, 2026

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit d735dc9.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 3 possible false positives for unstable benchmarks that are known to sometimes produce them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments
@raulcd raulcd raulcd approved these changes
@jonkeane jonkeane Awaiting requested review from jonkeane jonkeane is a code owner

Assignees

No one assigned

Labels

awaiting merge Awaiting merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@thisisnic @raulcd